

A slice of Python 


def a_function ( ) : 

# a comment 

a = "a string variable" 


class A_Class() : 

def init (self) : # Initializer 

self. xx =42 # instance var 


for obj in objects: 

# do something with obj 


What I inherited 


C : \Tools\Maya\Scripts 
C : \Tools\Maya\3rdParty\Scripts 



What I inherited 


C : \Tools\Maya\Scripts\MyExportInt . py 
C : \Tools\Maya\3rdParty\Scripts 


WTF? 



What I inherited 


# This is dumb. 

if not ValidateArray (objs) : 
for obj in objs: 

# Do stuff 

# This is all that is needed, 
for obj in objs: 

# Do Stuff 


Legacy code can be a burden 



Symptoms: confusing logic 


for f in files : 

if [f for s in GAME_EXT if s in f] : 
for s in GAME EXT: 




Symptoms: How to make changes? 
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Symptoms: Fear of breakage 




Bad code is an infection that spreads! 



Not all hope is lost 



Not all hope is lost 




Not all hope is lost 







The Prescription 



The Prescription 
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The Prescription: 
Continuous Maintenance 
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Bad Code is an infection that spreads 
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Bad Code is an infection that spreads 


How bad code happens 
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New to the language/system/toolchain 

etc. 







New to the language/system/toolchain 

etc. 
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The criteria changes 
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Everyone has deadlines. 






Others will reference the code. 



Others will reference the code. 



Growing systems entrench bad code 




Growing systems entrench bad code 




Overview 
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You must fight this infection regularly 



Don't write bad code, write less 

optimal code 


Dangerous shortcuts 




Dangerous shortcuts 




Dangerous shortcuts 




Dangerous shortcuts 




Dangerous shortcuts 
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Future proofing 







Future proofing 



Future proofing: 
Compartmentalize assumptions 



This is bad 


turnSpeed = strength * 42.57 



This is better 


torsionRatio = 42.57 

turnSpeed = strength * torsionRatio 



Compartmentalize assumptions 


def calcTorsionRatio ( vehicle=None , 

weigh t=None) : 

# We'll leave the complex math for later, 
return 42 . 57 


turnSpeed = strength * calcTorsionRatio () 


Group things by responsibility 




Group things by responsibility 

def rigFace(): 

browL = "browLeft" 
browR = "browRight" 
cheekL = "cheekL" 
cheekR = "cheekR" 
upperLipL = "upperLipL" 
upperLipR = "upperLipR" 

• • • 

browLPos = getPosition ( browL ) 
browRPos = getPosition ( browR ) 
cheekLPos = getPosition ( cheekL ) 
cheekRPos = getPosition ( cheekR ) 
upperLipL = getPosition ( upperLipL ) 


Group things by responsibility 

def rigFace(): 

browL = "browLeft" 

browLPos = getPosition ( browL ) 

# Code to build left brow 

browR = "browLeft" 

browRPos = getPosition ( browR ) 

# Code to build right brow 

cheekL = "cheekL" 

cheekLPos = getPosition ( cheekL ) 

# Code to build left cheek 


Tutorials, on boarding, documentation 








Documentation 
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Documentation 


def updateEmitterCloud ( ) 


def updateCloudEmitter () 


Documentation 


def getCollision (obj ) : 

for child in listRelatives (obj ) : 
if child. name () == 
return child 


’ collision 


Documentation 


def getCollision (obj ) : 

> > > 

Return collision or None if not 
found. 

r r r 

for child in listRelatives (obj ) : 
if child. name () == 
return child 


' collision 


Good documentation lets new people 

see the whole system. 








Documentation 


class Vector (VectorN) : 

def init (self, *args, **kwargs) : 

if args : 

if len(args)==l and hasattr (args [0] , ' iter '): 

args = args[0] 

try: 

self . assign (args) 

except : 

if isinstance (args , _api.MPoint) and args.w != 1.0: 
args = copy . deepcopy (args) . car tesianize () 
if isinstance (args , _api.MColor) and args. a 
pass 

if isinstance (args , _api .MVector) : 
args = tuple (args) 


! = 1 . 0 : 


This is a dumb comment 


// increment i 



This is a good comment 

# Go through collision largest 


# to smallest 

for col in reversed (getCollision () ) 


This is a good comment 

# Get the short name without namespace 
name . r split ( ' | ' ,1) [-1] .rsplit( ' : ' ,1) [-1] 


This is kind of awkward 


shortNameNoNamespace = 

name . rsplit ( ,!)[-!]. rsplit ( ' : ' ,1) [-1] 



This is really awful 


shrtNameNoNs = 

name . rsplit ( ,!)[-!]. rsplit ( ' : ' ,1) [-1] 



So much nicer! 


# Get the short name without namespace 
name . r split ( ' | ' ,1) [-1] .rsplit( ' : ' ,1) [-1] 


Sometimes hacks are required 


X X 


X X 


XXX 
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X X 


X X 


X X 


X X 


X X 


X XX 


X XX 


XX X 


XX X 


X X 


X X 


WARNING : The buffer 

MUST be accessed 
directly , backwards , 
due to a bug in 
the API . . . 
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XX X 


X XX 
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X X X X X X 


XXX 



Refactor your code 
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Breakdown monolithic 
classes/functions 
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By single responsibilities 
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Separate functionality and gui 




Separate functionality and gui 
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Separate functionality and gui 

def onClick() : 

for obj in selected () 


# Make LODs 


Separate functionality and gui 

def onClick() : 

makeLODs ( selected ( ) ) 

def makeLODs (objects) : 

# do the real work 
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Draw to understand 




Draw to understand 






Draw to understand 
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Globals 
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Why globals can be bad 




Why globals can be bad 



Why globals can be bad 




Why globals can be bad 



Why globals can be bad 



Why globals can be good 


def a (obj ) : 

b(obj) # Pass obj along 
def b (obj ) : 

c(obj) # Just keep passing it along 

• • • 

def g(obj) : 

h(obj) # And passing it some more 
def h (obj ) : 

# Finally do something with obj 
size = obj. size 


G l oba l Scope Issues 


class Exporter () : 

def init (self) : 

self .meshes = [] 
self .collision = [] 


def a (self) : 

# meshes and collision set here 


def h (self ) : 

# meshes and collision used here 


G l oba l Scope Issues 


def prepMesh ( . . . , exportTop ) : 

# exportTop not used here 

def combineCollision ( . . . , exportTop ) : 

# exportTop not used here 

def generateLOD ( . . . , exportTop ) : 

# exportTop not used here 

def resampleAnimation ( . . . , exportTop ) 

# exportTop not used here 



# exportTop not used here 



def combineCollision ( . . . , exportTop 

# exportTop not used hei 


def generateLO 
# exportT 


def resampl 

# exportTop not us4d here 


exportTop 




G l oba l Scope Issues 


class ExporterManager ( ) : 
def init (self) : 

# Easy to access as needed 

self . outputFiles = [] 

def determineOutput (self ) : 

# Clear setting mechanism 

# set self . outputFiles 
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Automate the testing 




Automate the testing 



Unit tests 



Unit tests 


def test_returnsNoneWithNoCollision () 
def test_findsValidCollision () : 
def test errorOnMultipleCollision () : 



Automated testing 

pendown 
right 90 
draw 1 5 
right 90 
draw 12 
right 90 
draw 1 5 
right 90 
draw 1 2 



Automated testing 



Super Automated Testing 



Overview 
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Bad code is an infection that spreads 




Fight the infection with regular 

maintenance. 



Fight the infection with regular 

maintenance. 
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The principles of good code are also 

ones of testable code. 


Testable Code = Good Code 



Resources 


The Art of Readable Code 

Dustin Boswell and Trevor Foucher 

Working Effectively with Legacy Code 

Michael Feathers 

Dive Into Python 

Mark Pilgrim 

(esp. Ch 7.3 on testing) 




patc@arena.net 


The Art of Readable Code 

Dustin Boswell and Trevor Foucher 


Working Effectively with Legacy Code 


Michael Feathers 


Dive Into Python 

Mark Pilgrim 




(esp. Ch 7.3 on testing) 



